testsuite: Add an inset shadow rendering test case

gl renderer: Cache current scale in RenderOpBuilder

We call ops_get_scale a lot, so this should be better. It will also make
a later x/y split for the scaling easier.

cssshadowvalue: Simplify inset shadow nodes to color nodes if possible

inset shadows like this are frequently used to implement highlights in
CSS without crowing them min size of the element.

gl renderer: Save one level of indentation

window: Don't set invalid csd input shape

This might happen if the CSS values of the decoration node are broken,
e.g. if people *accidentally* type large negative values for the
margins.

Fixes #2268

snapshot: Reorder color matrix nodes containing a transform node

A color matrix node that contains a transform node can also be expressed
the other way around, as a transform node containing a color matrix
node.

In the general case, the color matrix node will have to draw its
child to a texture so it can color every pixel of that texture, but the
renderers can short-cut this if the child of the color matrix node is
already a texture node. So if we have a node tree like

Color Matrix
    - Transform
        - Texture

The renderer would have to either check the grandchild of the color
matrix or simply fall back to rendering the transform node to a texture.

In the new configuration:

Transform
    - Color Matrix
        - Texture

The renderer can easily see that the child node of the color matrix node
is a texture, and skip rendering it to a texture.

This is for example happening in current Adwaita for spinners, which are
rotated symbolics.

snapshot: Factor out color matrix merging

We will use that elsewhere in the next commit.

gl renderer: Implement blurred shadow nodes

gl renderer: Take a TextureRegion in blur_texture

So we can refer to textures on a texture atlas.

gl renderer: Scale blur radius

We scale the texture size, so as a consequence we have to scale the blur
radius.

gl renderer: Add some often needed debug code

gl renderer: Blur shader improvements

Use a two-pass blur shader, fix a few other things and unify the
blurring of blur nodes and blurred outset shadow nodes.

Related to #1283

testblur: Fix default blur radius

sync it up with the adjustment value. also just add the destroyed
handler.

Merge branch 'improve-popover-menu-style' into 'master'

Adwaita: Improve popover.menu styling

See merge request GNOME/gtk!1083

Adwaita: Remove unused popover check/radio styling

Adwaita: Flatten circular buttons in popover.menu

Just reuse the list button style.

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Adwaita: Make check & radio symbolic icon style in popover.menu

The disabled modelbutton color needs to be specified explicitly
in order for check & radio to properly read the color with
currentColor.

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Adwaita: Improve popover.menu styling

Based on the mockup:
https://gitlab.gnome.org/Teams/Design/os-mockups/blob/master/menus/menu-design-patterns.png

- Remove horizontal padding from popover.menu contents
- Remove roundness from modelbutton
- Adjust sizing to match the mockup
- Fix visual glitches on button.image-button.model
- Add missing mergin to arrow icons
- Fix accelerator position in rtl direction
- Fix accelerator color

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Merge branch 'wip/muktupavels/opaque-region-master' into 'master'

window: fix opaque region

See merge request GNOME/gtk!1222

Merge branch 'sass-no-extend-compound-master' into 'master'

themes: remove various @extend with compound selectors

See merge request GNOME/gtk!1223

themes: remove various @extend with compound selectors

They are no longer supported by sass and broken with libsass 3.6.3
(https://github.com/sass/libsass/issues/3033)

This removes them by replacing them with a placeholder selector. This at
least brings the resulting CSS size down a bit so gtk can be build
again.

`%button.flat.suggested-action` has been replaced by
`%selection_mode_button_flat`, which is a more appropriate selector for
`.selection-mode button.titlebutton`.

The CSS was generated with libsass 3.5.5.

Co-authored-by: Christoph Reiter <reiter.christoph@gmail.com>
See https://gitlab.gnome.org/GNOME/gtk/issues/2237

window: fix opaque region

Decoration node for drawing is used only for client side decorated
windows, but corners from opaque region is subtracted also for
normal windows.

Rename function to better reflect what it does and do not subtract
corners if decoration node was not used for drawing.

Merge branch 'gtk-link-color' into 'master'

Adwaita: always use link specific colors for link selector

See merge request GNOME/gtk!1201

Merge branch 'matthiasc/for-master' into 'master'

some builder-tool tests

See merge request GNOME/gtk!1220

Update the test scripts

All tools got renamed to have a gtk4 prefix.

Add a test for the previous fix

Merge branch 'matthiasc/for-master' into 'master'

builder-tool: Fix handling of layout properties

See merge request GNOME/gtk!1219

builder-tool: Fix handling of layout properties

We were not properly identifying layout properties
that need to be kept, causing GtkGrid left-attachment
to go missing in widget-factory.ui.

Fix up widget-factory ui

Merge branch 'dont-color-labels-directly-gtk4' into 'master'

Adwaita: Avoid directly coloring labels wherever possible (GTK4)

Closes #1643

See merge request GNOME/gtk!1209

reftests: Fix css-multi-state test

Properly unset theme CSS and use the right selector for the toggle
buttons.

Adwaita: always use link specific colors for link selector

link target uses `$selected_bg_color` and `$selected_fg_color` for the
definition of some selectors (e.g. selected).

While `$link_color` and `$link_visited_color` are based on
`$selected_bg_color`, this does not seem coherent.

Replacing `$selected_*g_color` in link target style with link colors
defined in `_colors.scss file`.

widget-factory: Fix a popover menu

This one is manually constructed, and things have
changed enough for that to break. Fix it up.

gl renderer: Don't use g_assert() for code that should always run

g_assert can be compiled out.

gl renderer: Don't save repeat nodes to disk

This is just debuggin code someone forgot.

Merge branch 'single-compilation-symbol' into 'master'

Use a single compilation symbol

See merge request GNOME/gtk!1216

Merge branch 'fix-issue-1646-master' into 'master'

HighContrast: Add missing color to entry

Closes #1646

See merge request GNOME/gtk!1215

Use a single compilation symbol

We use a compilation symbol in our build to allow the inclusion of
specific headers while building GTK, to avoid the need to include only
the global header.

Each namespace has its own compilation symbol because we used to have
different libraries, and strict symbol visibility between libraries;
now that we have a single library, and we can use private symbols across
namespaces while building GTK, we should have a single compilation
symbol, and simplify the build rules.

HighContrast: Add missing color to entry

So entries don't inherit color from the parent element anymore.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1646

Merge branch 'broken-annotation' into 'master'

builder: Fix broken annotation

See merge request GNOME/gtk!1213

builder: Fix broken annotation

Missing a ':', was introduced in 0627bba5a0d.

gl renderer: Load flipped-y offscreen vertex data directly

Instead of loading the unflipped version first and then flipping it.
Don't do it in add_render_ops either but only in the function actually
adding the render ops for the nodes, since those frequently have
early-out conditions that don't need the vertex data at all.

testsuite: Add another repeat node test case

gl renderer: Fix repeat node shader once again

gl renderer: Care more about 0×0 offscreen nodes

These never result in a rendering of course, but we must make sure not
to create a 0×0 texture for them, since that will cause problems later
on.

Merge branch 'unsafe-strdup-printf' into 'master'

gtk-demo: Fix incorrect use of g_strdup_printf

See merge request GNOME/gtk!1211

gtk-demo: Fix incorrect use of g_strdup_printf

Was broken in a refactoring in 4a2f313

Adwaita: Avoid directly coloring labels wherever possible

Directly coloring labels can create a lot of unintended behaviors.
Unfortunately we can't avoid directly coloring `label:disabled` because
it can be used individually. This, however, tries to inherit color from
the parent element of labels wherever possible.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1643

testsuite: Fix expected test output

This reverts part of 91497348bf1ea90ef1056bdc8727bbea645c21e3

cssprovider: Don't add empty rulesets

This change was lost during the parser changes, but the newly added test
exposes it.

And fix the expected error, the syntax has changed.

Fix crash in parse_border()

The bug was introduced in commit:
9b7640b8 by Benjamin Otte, 2012-03-26 17:24:02
styleproperty: Make _gtk_style_property_parse_value() return a CssValue

In that commit, `values` changed from `GValue*` to `GtkCssValue**`,
but one `!G_IS_VALUE (&values[8])` was left untouched. As a result,
if `border` shorthand contains anything after color, it might crash,
depending on memory layout.

New test included.

Fixes: #751

Merge branch 'wip/otte/builder' into 'master'

Reorganize various GtkBuilder things

See merge request GNOME/gtk!1204

builder: Make the struct private

We don't want anyone to subclass it - if we want to allow bindings to
extend GtkBuilder, we will be using delegate objects/functions for it.

builder: Remove get_type_from_name() vfunc

builder: Improve signal connecting

- Propagate the error back to the parser, so we get a proper GError
  instead of a g_warning().
- Connect closures by id, don't construct a name from the ids so that
  glib can take it apart again.

builder: Connect signals automatically

gtk_builder_connect_signals() is no longer necessary, because all the
setup that made it necessary to have this extra step is now done
automatically via the closure functions.

reftests: Switch to gtk_builder_set_closure_func()

widget: Change templates to use closure functions

... instead of connect functions.

builder: Add gtk_builder_set_closure_func()

This allows bindings and templates to setup a way to create closures.

widget: Reinstate custom closure func

It turns out it ws used in various places.

builder: Add gtk_builder_create_closure()

This will be the future way to connect signals automatically (and be
used for other things, too).

For now, gtk_builder_connect_signals_default() is ported to use it.

builder: Remove user_data argument from gtk_builder_connect_signals()

This is pretty unused and gets in the way of the next steps.

A potential side effect is that for templates the widget was passed as
the user data argument. If that turns out to be important, we have to
special case that situation.

builder: Remove application setter/getter

It's unused and doesn't belong to GtkBuilder.

builder: Carry the module as part of the GtkBuilder object

This way, we don't open the module in every place we want to look up
functions.

builder: Fix typo in docs

testsuite: Print useful statement on notify test error

... and don't instantly abort, test the rest of the properties, too.

types: Remove and unused typedef

GtkRcPropertyParser is gone.

settings: Remove RC property parsers

They've not been used for a long time and the APIs where they could
reasonably be used have been gone for as long.

settings: Remove code dealing with property parsers

They only remained in private code where they were always set to NULL.

adwaita: level bar refinements
 - shave a few pixels off large level bar
 - thinner discrete level bar

gl renderer: Remove unused function

testsuite: Add a testcase for repeat nodes that don't repeat

gl renderer: Add a short cut for repeat nodes that don't repeat

These happen and we don't even need to draw the child to a texture.

gl renderer: Fix repeat nodes that don't repeat

Fixes #2234

csstokenizer: Plug a leak

rendernodeparser: Only report a GError* if we have one

rendernodeparser: Handle resolving NULL urls

docs: Add migration note for iconify → minimize

See commit b70e4f6de95b.

Updated Spanish translation

Merge branch 'gdk-backend-imports' into 'master'

Correct Gdk backend import paths and related gtk-doc

See merge request GNOME/gtk!1197

Correct Gdk backend import paths and related gtk-doc

Correct the paths in the include guard warnings and use the `@Include`
directive for the Gdk backends.

Merge branch 'seal-popover-menu' into 'master'

Privatize popover menu pieces

See merge request GNOME/gtk!925

Merge branch 'signal-naming' into 'master'

gtkstyleprovider: Rename -gtk-style-changed signal to have a valid name

See merge request GNOME/gtk!1176

Merge branch 'issue-67' into 'master'

Rename "iconify" to "minimize"

Closes #67

See merge request GNOME/gtk!1190

Merge branch 'meson-bump' into 'master'

Bump up the required version of Meson

See merge request GNOME/gtk!1191

Disable GdkSurface.minimize on Wayland

The existing xdg-shell protocols do not support minimization in a way
that allows us to implement the GdkSurface API; the only minimization
operation does not come with a state notification, nor it comes with
a way to undo itself.

Closes: #67

Rename "iconify" to "minimize"

The "iconified" state is mostly an X11-ism; every other platform calls
this state "minimized" because it may not involve turning a window into
an icon at all.

Bump up the required version of Meson

Match the one we're testing on our CI infrastructure.

Merge branch 'gdkx-import-doc' into 'master'

Correct GdkX11 import path in docs and include guards

Closes #2254

See merge request GNOME/gtk!1188

Correct GdkX11 import path in docs and include guards

Update the include directives in the documentation, as well as the
include guards in headers, to point to gdk/x11/gdkx.h.

closes #2254

Merge branch 'fix-menu-check-radio-styling-gtk4' into 'master'

Adwaita: Fix check/radio styling in menu (GTK4)

See merge request GNOME/gtk!1187

Merge branch 'dont-hardcode-shared-library' into 'master'

build: Don't hardcode shared_library() in meson

See merge request GNOME/gtk!1172

Try building static libs on CI

Adwaita: Fix check/radio styling in menu

The new check/radio style increased their specificity, but it was not
synchronized in the menu styling.

This commit increases the specificity to match the updated check/radio
styling.

https://gitlab.gnome.org/GNOME/gtk/issues/2096

Merge branch 'building-docs' into 'master'

Document the use of build types when configuing GTK

See merge request GNOME/gtk!1113

Merge branch 'Master_checkradio_refresh' into 'master'

Refresh check/radio styling, ported from gtk3

See merge request GNOME/gtk!1175

Refresh check/radio styling, ported from gtk3

Merge branch 'fix-emoji-picker-styling' into 'master'

Adwaita: Fix emoji picker styling (GTK4)

See merge request GNOME/gtk!1181

Adwaita: Fix emoji picker styling

- Adjust the emoji picker appearance to be the same as gtk3's.
- Fix button.emoji-section margins in RTL.
- Prevent the .emoji hover effect from being applied to the entire
  nested popover.

headerbar: Disconnect signal handlers when children are removed

The signal handler that calls notify_child_cb() is disconnected
from a child widget before the child is removed from the header bar.
gtk_header_bar_dispose() chains up before destroying the start and end
boxes, thus avoiding calls to notify_child_cb() after start_box and
end_box have been cleared.

Fixes #2246